Bare minimum self-checks

What do you think of a person who only does the bare minimum?

• I've updated this PR with the latest code from main
• I've done a cursory QA pass of my code locally
• I've ensured all automated status check and tests pass
• I've connected this PR to an issue

Pieces of flare

• I've written a unit or functional test for my code
• I've updated relevant documentation it my code changes it
• I've updated this repo's README if my code changes it
• I've updated this repo's CHANGELOG with my change unless its a trivial change (like updating a typo in the docs)

Finally

• I've requested a review with relevant people

If you have any issues or need help please join the #contributors channel in the Lando slack and someone will gladly help you out!

You can also check out the coder guide.

Description:

This PR fixes a bug where the pr-docs-tests.yml workflow was not saving the VitePress doc cache on failed runs, leading to degraded CI performance. The save-always: true input for actions/cache@v5 is deprecated and non-functional. This change replaces the single cache step with separate actions/cache/restore and actions/cache/save steps, ensuring the cache is always saved using if: always().